Gemfile

@@ -77,6 +77,7 @@ gem 'rufus-scheduler', '~> 3.0.8', require: false

                 gem 'sass-rails',   '~> 4.0.0'
              

                 gem 'select2-rails', '~> 3.5.4'
              

                 gem 'spectrum-rails'
              

                +gem 'string-scrub'	# for ruby <2.1
              

                 gem 'therubyracer', '~> 0.12.1'
              

                 gem 'typhoeus', '~> 0.6.3'
              

                 gem 'uglifier', '>= 1.3.0'
              

Gemfile.lock

@@ -371,6 +371,7 @@ GEM

                       actionpack (>= 3.0)
              

                       activesupport (>= 3.0)
              

                       sprockets (~> 2.8)
              

                +    string-scrub (0.0.5)
              

                     systemu (2.6.4)
              

                     term-ansicolor (1.3.0)
              

                       tins (~> 1.0)
              
@@ -512,6 +513,7 @@ DEPENDENCIES

                   spectrum-rails
              

                   spring
              

                   spring-commands-rspec
              

                +  string-scrub
              

                   therubyracer (~> 0.12.1)
              

                   tumblr_client
              

                   twilio-ruby (~> 3.11.5)
              

app/models/agent_log.rb

@@ -11,6 +11,7 @@ class AgentLog < ActiveRecord::Base

                   validates_presence_of :agent, :message
              

                   validates_numericality_of :level, :only_integer => true, :greater_than_or_equal_to => 0, :less_than => 5
              

                 
              

                +  before_validation :scrub_message
              

                   before_save :truncate_message
              

                 
              

                   def self.log_for_agent(agent, message, options = {})
              
@@ -31,6 +32,13 @@ class AgentLog < ActiveRecord::Base

                 
              

                   protected
              

                 
              

                +  def scrub_message
              

                +    if message_changed?
              

                +      self.message.scrub!{ |bytes| "<#{bytes.unpack('H*')[0]}>" }
              

                +    end
              

                +    true
              

                +  end
              

                +
              

                   def truncate_message
              

                     self.message = message[0...10_000] if message.present?
              

                   end
              

spec/models/agent_log_spec.rb

@@ -1,3 +1,4 @@

                +# -*- coding: utf-8 -*-
              

                 require 'spec_helper'
              

                 
              

                 describe AgentLog do
              
@@ -42,6 +43,13 @@ describe AgentLog do

                     end
              

                   end
              

                 
              

                +  it "replaces invalid byte sequences in a message" do
              

                +    log = AgentLog.new(:agent => agents(:jane_website_agent), level: 3)
              

                +    log.message = "\u{3042}\xffA\x95"
              

                +    expect { log.save! }.not_to raise_error
              

                +    expect(log.message).to eq("\u{3042}<ff>A\<95>")
              

                +  end
              

                +
              

                   it "truncates message to a reasonable length" do
              

                     log = AgentLog.new(:agent => agents(:jane_website_agent), :level => 3)
              

                     log.message = "a" * 11_000